-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Monopole SKIM preparation for Run 3 #45134
Monopole SKIM preparation for Run 3 #45134
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45134/40474
|
A new Pull Request was created by @ThOliveira23 for master. It involves the following packages:
@AdrianoDee, @miquork, @cmsbuild, @sunilUIET can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
-1 Failed Tests: UnitTests Unit TestsI found 1 errors in the following unit tests: ---> test test-das-selected-lumis had ERRORS Comparison SummarySummary:
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2876f5/39707/summary.html Comparison SummarySummary:
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45134/40522
|
Pull request #45134 was updated. @miquork, @cmsbuild, @AdrianoDee, @sunilUIET can you please check and sign again. |
Dear @cms-sw/pdmv-l2 could you kindly review and sign this PR. We would like to have it for the restart of the pp data taking starting period. |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2876f5/39788/summary.html Comparison SummarySummary:
|
+Upgrade |
Hi @cms-sw/pdmv-l2, could you please sign this PR? |
+pdmv |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Provide an update to the trigger paths to be employed in the MONOPOLE SKIM file for the Run 3. The additional inclusion of trigger paths associated with MET can be mentioned.
Additionally, the inclusion of the EXOMONOPOLE SKIM file to be executed in the MET and JetMET datasets for the Run 3.
PR validation: The PR was validated through the the basic test procedure available in the CMSSW PR instructions
FYI: @malbouis, @srimanob, @CarstenHensel